Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pipes-Mariana-Hotel #26

Open
wants to merge 16 commits into
base: master
Choose a base branch
from
Open

Conversation

lmarianarp19
Copy link

Hotel

Congratulations! You're submitting your assignment!

Comprehension Questions

Question Answer
Describe a design decision you had to make when working on this project. What options were you considering? What helped you make your final decision? I decided to create at Admin class to control the reservations for individual rooms and blocks, I need this because I need something to control the reservations. At the beginning I thought was easier creating class variables instead.
Describe a concept that you gained more clarity on as you worked on this assignment. How to work in a class with methods from a different class.
Describe a nominal test that you wrote for this assignment. To see if it is possible to make a reservation when I know there is extra rooms.
Describe an edge case test that you wrote for this assignment. To expect an error if there is not rooms available for an specific date range.
How do you feel you did in writing pseudocode first, then writing the tests and then the code? It took me a lot of effort because sometimes I was no sure how to start writing the pseudocode. Doing first the test was also difficult because I was no sure about the names to use.

@PilgrimMemoirs
Copy link

Hotel

What We're Looking For

Feature Feedback
Baseline
Used git regularly Need to commit more often
Answer comprehension questions Well Done
Design
Each class is responsible for a single piece of the program Well Done
Classes are loosely coupled Well Done - nice use of splitting up functionality into many mehthods
Wave 1
List rooms Well Done
Reserve a room for a given date range Well Done
List reservations for a given date Well Done
Calculate reservation price Well Done - method name totalcost should have an underscore between the two words in the name, total_cost.
Invalid date range produces an error Well Done
Test coverage Well Done
Wave 2
View available rooms for a given date range Well Done
Reserving a room that is not available produces an error Well Done
Test coverage Well Done
Wave 3
Create a block of rooms Well Done
Check if a block has rooms Well Done
Reserve a room from a block Well Done
Test coverage Well Done
Additional Feedback
Nice work splitting functionality between methods and using them to call each other.
Well organized tests, appropriate use of before do in each respected describe block.
Submission meets projects expectations, nice work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants